Swift: explicitly cast UInt32 to dispatch_flags_t
#445
Conversation
Because `dispatch_flags_t` is a sized enumeration on Windows, the type is not implicitly constructed (as in C/C++). Add the explicit constructor call, which should be portable to all the targets.
|
@swift-ci please test |
|
@ktopley-apple - this fixes the Windows build and should work across all the targets |
|
I'm concerned about what type is being picked for the |
dispatch_block_flags_t only has 6 defined bits, all at the low-order end, so in reality this should not be a concern. We have the practical problem that the raw value for this type is not the same on all platforms (and not even on all versions of the same platform) and there seems to be no other way to write one line of code that will convince the compiler to do the "right thing". |
|
The build failed. I saw this same failure when I last tried to fix this in a similar way. I filed a Radar on the compiler for that. It looks like the only way around this for now is to put back the Win32 conditional and use the previous version of the code for the Windows build. |
|
Closing this since it doesn't work in favour of #446 |
Because
dispatch_flags_tis a sized enumeration on Windows, the typeis not implicitly constructed (as in C/C++). Add the explicit
constructor call, which should be portable to all the targets.